Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

HAML Templates: Pro or Con

#1

Zachinglis (http://github.com/zachinglis/res... has put in a patch to add HAML templates. I think this should be pointed to and not pulled into mainline: generated code is a bitch to maintain, and anyone submitting code is going to only want to make & debug changes to one or the other template system.

On the other hand, the change is clean, noninteracting and only matters during the generator phase.

So: +1 if you think this should be pulled in, -1 if you think a standalone patch is appropriate. And if you have a more pragmatic solution do tell.

Reported by mrflip · August 8th, 2008 @ 03:17 PM

State: hold
Milestone: none
Assigned to: mrflip mrflip

Activity

  1. mrflip
    mrflip
    • State changed from new to hold

    August 14th, 2008 @ 12:27 PM

  2. mrflip
    mrflip

    I've thought about this more, and if Zach (or anyone) is willing to move the HAML views into their own tiny little generator that would work. A separate generator within the same plugin decouples development, sits out of the way enough to avoid confusing n00bs, but gives everyone HAML views free with purchase.

    • make a new generator directory generators/authenticated_views_haml
    • copy just the HAML files into generators/authenticated_views_haml/templates/views
    • make an authenticated_views_haml_generator.rb to install them

    If you're not zachinglis, and you're interested in doing this, visit his fork on github: http://github.com/zachinglis/res...

    August 17th, 2008 @ 06:46 PM

  3. Zach Inglis
    Zach Inglis

    In response to making this a separate generator, I am up for that.

    August 17th, 2008 @ 06:48 PM

  4. Lar Van Der Jagt
    Lar Van Der Jagt

    +1 both on this being pulled in & for the optional generator approach.

    It looks like this is in Zach's fork already, can we get it merged in properly?

    August 19th, 2008 @ 02:31 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.